-
Notifications
You must be signed in to change notification settings - Fork 242
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add a guides for generating embeddings with Mistral, OpenAI, Voyage, and Cloudflare #2965
Conversation
How to see the preview of this PR?Go to this URL: https://website-git-deploy-preview-mei-16-meili.vercel.app/docs/branch:embedding-guides Credentials to access the page are in the company's password manager as "Docs deploy preview". |
Co-authored-by: Louis Dureuil <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM apart from the version requirement and small nitpicks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @qdequele, I request some minor changes.
Also, gui would likely request to remove mentions to "simply" and other wording that suggest the process is easy. I would agree with this advice :p
@Strift I finally added some pages about how to do the same with others third party tools :) |
70800f6
to
fcf2f4e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @qdequele, I have sent minor suggestions. Please accept them all :)
The changes are:
- use relative links (best for reader + tooling)
- add UTM parameters
- "embedding" -> "embeddings"
Then, we're good to merge. The articles can be improved later, but it would be a shame to delay publishing because they're already good enough as-is, IMO.
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
Co-authored-by: Laurent Cazanove <[email protected]>
All fixed :) |
I created a first guide to use Mistral as an embedder in Meilisearch. Once this is confirmed I would be able to duplicate this content for all the embedders (OpenAI, Voyage, Cloudflare, and others).